Skip to content

fix: add constants required for project navigation movement to global config#1006

Merged
AbhishekA1509 merged 3 commits into
mainfrom
fix/ea-project
Jan 21, 2026
Merged

fix: add constants required for project navigation movement to global config#1006
AbhishekA1509 merged 3 commits into
mainfrom
fix/ea-project

Conversation

@AbhishekA1509

Copy link
Copy Markdown
Member

Description

fix: add constants required for project navigation movement to global config

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

QA

Checklist

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@AbhishekA1509 AbhishekA1509 self-assigned this Jan 21, 2026
Copilot AI review requested due to automatic review settings January 21, 2026 06:05
vivek-devtron
vivek-devtron previously approved these changes Jan 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the "projects" navigation item from the Application Management section to the Global Configuration section, addressing a navigation structure issue.

Changes:

  • Moved navigation item ID from 'application-management-projects' to 'global-configuration-projects' in the type definitions
  • Updated URL constant from APPLICATION_MANAGEMENT_PROJECTS to GLOBAL_CONFIG_PROJECTS with the new path
  • Bumped package version from 1.22.6 to 1.22.7

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
src/Pages-Devtron-2.0/Navigation/types.ts Removed 'application-management-projects' from NavigationItemID type and added 'global-configuration-projects'
src/Common/Constants.ts Removed APPLICATION_MANAGEMENT_PROJECTS URL constant and added GLOBAL_CONFIG_PROJECTS pointing to global-configuration/projects
package.json Version bump from 1.22.6 to 1.22.7
package-lock.json Version bump from 1.22.6 to 1.22.7 in lockfile

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AbhishekA1509 AbhishekA1509 merged commit 871cedb into main Jan 21, 2026
9 checks passed
@AbhishekA1509 AbhishekA1509 deleted the fix/ea-project branch January 21, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants